Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix test failure when building OTP in a path with special characters #6344

Merged
merged 2 commits into from
Dec 19, 2024

Conversation

miklcct
Copy link
Contributor

@miklcct miklcct commented Dec 18, 2024

Summary

This pull request fixes test failures when building OpenTripPlanner in a folder containing URL-encodable characters (such as space, Chinese characters, etc.).

Please refer to https://stackoverflow.com/a/13470643 for details.

Issue

fixes #6337

Unit tests

Updated

Documentation

None

Changelog

Not needed

Bumping the serialization version id

Not needed

@miklcct miklcct requested a review from a team as a code owner December 18, 2024 11:59
@leonardehrenfried leonardehrenfried changed the title fix test failure when building OTP in a path with special characters. Fix test failure when building OTP in a path with special characters Dec 18, 2024
Copy link

codecov bot commented Dec 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.85%. Comparing base (2d012a3) to head (2eedb12).

Additional details and impacted files
@@              Coverage Diff              @@
##             dev-2.x    #6344      +/-   ##
=============================================
- Coverage      69.85%   69.85%   -0.01%     
+ Complexity     17922    17920       -2     
=============================================
  Files           2035     2035              
  Lines          76495    76495              
  Branches        7824     7824              
=============================================
- Hits           53435    53434       -1     
  Misses         20324    20324              
- Partials        2736     2737       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tkalvas tkalvas merged commit 4198bb6 into opentripplanner:dev-2.x Dec 19, 2024
5 checks passed
@miklcct miklcct deleted the fix/build_path branch December 19, 2024 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Test failure (file not found) trying to build on my new machine
3 participants